From: Keir Fraser <keir.fraser@citrix.com>
Date: Mon, 22 Sep 2008 14:15:19 +0000 (+0100)
Subject: vsprintf: Better size_t input checking, and fix return values of
X-Git-Tag: archive/raspbian/4.8.0-1+rpi1~1^2~14101^2~43
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=2d4c3c5c9c2c4ebdd4c8b1de3807247f75904376;p=xen.git

vsprintf: Better size_t input checking, and fix return values of
[v]scnprintf() when size <= 0.

Signed-off-by: Keir Fraser <keir.fraser@citrix.com>
---

diff --git a/xen/common/vsprintf.c b/xen/common/vsprintf.c
index 5ee5f4efce..8fd1ac25de 100644
--- a/xen/common/vsprintf.c
+++ b/xen/common/vsprintf.c
@@ -272,7 +272,7 @@ int vsnprintf(char *buf, size_t size, const char *fmt, va_list args)
                                 /* 'z' changed to 'Z' --davidm 1/25/99 */
 
     /* Reject out-of-range values early */
-    BUG_ON((int)size < 0);
+    BUG_ON(((int)size < 0) || ((unsigned int)size != size));
 
     str = buf;
     end = buf + size - 1;
@@ -504,8 +504,10 @@ int vscnprintf(char *buf, size_t size, const char *fmt, va_list args)
 {
     int i;
 
-    i=vsnprintf(buf,size,fmt,args);
-    return (i >= size) ? (size - 1) : i;
+    i = vsnprintf(buf,size,fmt,args);
+    if (i >= size)
+        i = size - 1;
+    return (i > 0) ? i : 0;
 }
 
 EXPORT_SYMBOL(vscnprintf);
@@ -772,7 +774,9 @@ int scnprintf(char * buf, size_t size, const char *fmt, ...)
     va_start(args, fmt);
     i = vsnprintf(buf, size, fmt, args);
     va_end(args);
-    return (i >= size) ? (size - 1) : i;
+    if (i >= size)
+        i = size - 1;
+    return (i > 0) ? i : 0;
 }
 EXPORT_SYMBOL(scnprintf);